Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Define IDL instructions for the confidential transfer fee extension #51

Merged
merged 1 commit into from
Nov 14, 2024

Conversation

0xCipherCoder
Copy link
Contributor

This PR manually updates the Codama IDL to support all instructions in the confidential transfer fee extension. This allows helpers to be generated for the new web3.js.
This pull request was created for https://app.gib.work/bounties/a487ebf9-3e05-4877-920d-23e7d37f351e in an attempt to solve a bounty #23 . Payment for the bounty is immediately sent to the contributor after merge.

@0xCipherCoder 0xCipherCoder marked this pull request as draft November 14, 2024 09:04
@0xCipherCoder
Copy link
Contributor Author

@lorisleiva I have added all instructions in IDL and added one test file for the initialization of the transfer fee. You can review this part. For other instruction testing taking time to understand ZK involved for multiple instructions that does not have any references in code. Let me know if have any references to check.

For example -

/// In order for this instruction to be successfully processed, it must be
/// accompanied by the `VerifyCiphertextCiphertextEquality` instruction
/// of the `zk_elgamal_proof` program in the same transaction

@lorisleiva
Copy link
Member

Thank you! This extension is very complex so, for now, I'm happy with just the initialisation test. 👌 I'll be reviewing this in a sec.

Copy link
Member

@lorisleiva lorisleiva left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great job! Thank you!

@lorisleiva lorisleiva marked this pull request as ready for review November 14, 2024 11:47
@lorisleiva lorisleiva merged commit 8dbae66 into solana-program:main Nov 14, 2024
5 checks passed
@0xCipherCoder
Copy link
Contributor Author

Great job! Thank you!

Thank you for all the support! @lorisleiva

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants